name: pre-commit run_id: commands[2] env HOME: /home/jenkins env LANG: C.UTF-8 env PATH: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit/bin:/opt/pyenv/bin:/tmp/venv-eon5/bin:/opt/pyenv/shims:/home/jenkins/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/opt/puppetlabs/bin env PYTHONHASHSEED: 163285443 env PIP_DISABLE_PIP_VERSION_CHECK: 1 env PYTHONIOENCODING: utf-8 env OLM_TIMER1: 3000 env OLM_TIMER2: 2000 env TOX_ENV_NAME: pre-commit env TOX_WORK_DIR: /w/workspace/transportpce-tox-verify-transportpce-master/.tox env TOX_ENV_DIR: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit env VIRTUAL_ENV: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit metadata pid: 28955 cwd: /w/workspace/transportpce-tox-verify-transportpce-master/tests allow: /w/workspace/transportpce-tox-verify-transportpce-master/.tox/pre-commit/bin/*:sh:cpan:sudo:which:fixCIcentOS8reposMirrors.sh cmd: pre-commit run --all-files --show-diff-on-failure exit_code: 1 [WARNING] hook id `remove-tabs` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [WARNING] hook id `perltidy` uses deprecated stage names (commit) which will be removed in a future version. run: `pre-commit migrate-config` to automatically fix this. [INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks. [WARNING] repo `https://github.com/pre-commit/pre-commit-hooks` uses deprecated stage names (commit, push) which will be removed in a future version. Hint: often `pre-commit autoupdate --repo https://github.com/pre-commit/pre-commit-hooks` will fix this. if it does not -- consider reporting an issue to that repo. [INFO] Initializing environment for https://github.com/jorisroovers/gitlint. [INFO] Initializing environment for https://github.com/jorisroovers/gitlint:./gitlint-core[trusted-deps]. [INFO] Initializing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Initializing environment for https://github.com/pre-commit/mirrors-autopep8. [INFO] Initializing environment for https://github.com/perltidy/perltidy. [INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/Lucas-C/pre-commit-hooks. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/pre-commit/mirrors-autopep8. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... [INFO] Installing environment for https://github.com/perltidy/perltidy. [INFO] Once installed this environment will be reused. [INFO] This may take a few minutes... trim trailing whitespace.................................................Failed - hook id: trailing-whitespace - exit code: 1 - files were modified by this hook Fixing tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractTapiTopoToNbi.java Fixing tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/ORToTapiTopoConversionFactory.java Fixing tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractORTopoToNbi.java Tabs remover.............................................................Passed autopep8.................................................................Passed perltidy.................................................................Passed pre-commit hook(s) made changes. If you are seeing this message in CI, reproduce locally with: `pre-commit run --all-files`. To run `pre-commit` as part of git workflow, use `pre-commit install`. All changes made by hooks: diff --git a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractORTopoToNbi.java b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractORTopoToNbi.java index 6b011d35..0b0c455d 100644 --- a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractORTopoToNbi.java +++ b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractORTopoToNbi.java @@ -241,7 +241,7 @@ public class AbstractORTopoToNbi { } /** - * Provides a list of Photonic TapiNodes that include eODU NEPs + * Provides a list of Photonic TapiNodes that include eODU NEPs */ private List pruneTapiPhotonicNodes() { diff --git a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractTapiTopoToNbi.java b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractTapiTopoToNbi.java index d3b6d630..d3b7acf9 100644 --- a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractTapiTopoToNbi.java +++ b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/AbstractTapiTopoToNbi.java @@ -132,7 +132,7 @@ public class AbstractTapiTopoToNbi { } /** - * Provides a Map that includes only NEPs which correspond to pure photonic Nodes + * Provides a Map that includes only NEPs which correspond to pure photonic Nodes */ private Map pruneTapiRoadmNeps() { List tapiPhotonicNodes = this.tapiNodes.values().stream() diff --git a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/ORToTapiTopoConversionFactory.java b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/ORToTapiTopoConversionFactory.java index 3cf8535f..48530dbd 100644 --- a/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/ORToTapiTopoConversionFactory.java +++ b/tapi/src/main/java/org/opendaylight/transportpce/tapi/topology/ORToTapiTopoConversionFactory.java @@ -155,7 +155,7 @@ import org.slf4j.LoggerFactory; * Provided methods are shared between Classes : * - dedicated to the generation of Topology on RPC requests, * - dedicated to the generation of Tapi Topology at initialization from existing Datastore - * - dedicated to Tapi Topology update on Data-Tree-Changes + * - dedicated to Tapi Topology update on Data-Tree-Changes */ public class ORToTapiTopoConversionFactory { @@ -353,7 +353,7 @@ public class ORToTapiTopoConversionFactory { * @param nodeUuid The Uuid of the node. * @param subNodeName The extension in OpenROADM NodeId that identifies the considered SRG/DEG. this subnode Name * is used to form the name of the NRG, as the information of the extension is removed from TAPI - * nodeID (no disaggregation of ROADM in Tapi topology representation). + * nodeID (no disaggregation of ROADM in Tapi topology representation). * @param onepl List of owned node edge points to be considered in the NRGs. * @param forwardingRule Forwarding Rule set in the NRG (May or Cannot Forward). * @param index Index used to form the SRG name. @@ -1153,7 +1153,7 @@ public class ORToTapiTopoConversionFactory { * Adds the Payload Structure and the Photonic Node Edge Point Spec to an OwnedNodeEdgePointBuilder * @param nodeId OpenROADM node Id * @param freqMap Map of Min/Max Frequency corresponding to the different occupied-slots low/high boundaries. - * @param operModeList List of Keys of the operational modes supported by the NEP + * @param operModeList List of Keys of the operational modes supported by the NEP * @param sicColl A collection of supported interface capabilities that the tp/NEP supports. * @param onepBldr The onepBuilder of the NEP to augment. * @param lpn Layer protocol qualifier that the tp/NEP supports.